Skip to content

Remove legacy prompt/security-audit APIs after unified review migration#20

Merged
matej merged 1 commit intomainfrom
matej/remove-legacy-prompt-apis
Feb 7, 2026
Merged

Remove legacy prompt/security-audit APIs after unified review migration#20
matej merged 1 commit intomainfrom
matej/remove-legacy-prompt-apis

Conversation

@matej
Copy link
Copy Markdown
Member

@matej matej commented Feb 6, 2026

Summary

  • remove legacy prompt APIs: get_code_review_prompt, get_security_review_prompt, and get_security_audit_prompt
  • remove legacy security-audit aliases in runner/module (run_security_audit, _extract_security_findings)
  • keep a single unified prompt entrypoint and extract shared prompt helpers for diff/file-reading sections
  • update tests to target unified APIs (get_unified_review_prompt, run_code_review, _extract_review_findings)
  • update docs to describe unified prompt behavior for custom security instructions

Validation

  • pytest claudecode -v
  • ~/.bun/bin/bun test scripts/comment-pr-findings.bun.test.js

Notes

  • local untracked files .DS_Store and docs/header.svg were not included in this PR.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found. Changes look good.

@matej matej merged commit d242216 into main Feb 7, 2026
2 checks passed
@matej matej deleted the matej/remove-legacy-prompt-apis branch February 7, 2026 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant